src/main/java/com/example/solidconnection/term/domain/Term.java (1)

18-18: ⚠️ Potential issue | 🟠 Major

@AllArgsConstructor를 제거하여 null 변환 로직 우회 방지

현재 코드에서 @AllArgsConstructor가 Term(Long id, String name, Boolean isCurrent) 3개 인자 생성자를 자동 생성하고 있습니다. 이는 37-40번 줄의 커스텀 생성자에서 구현한 false → null 변환 로직을 완전히 우회할 수 있습니다.

1. 발견된 문제

   • @AllArgsConstructor의 3인자 생성자로 new Term(1L, "name", false)를 호출하면 isCurrent에 false가 그대로 저장됨
   • 34번 줄의 unique 제약에 의해 false 값이 하나만 허용되어 의도하지 않은 제약 위반 발생 가능

2. 현재 상태

   • 코드베이스에서는 2인자 생성자(new Term(name, isCurrent))만 사용 중
   • 3인자 생성자 직접 호출은 없음

3. 권장 조치

   • @AllArgsConstructor 제거 (34번 줄 삭제)
   • 2인자 커스텀 생성자와 setter 메서드만으로 충분

🔧 제안: `@AllArgsConstructor` 제거

 `@Getter`
 `@NoArgsConstructor`(access = AccessLevel.PROTECTED)
 `@Entity`
-@AllArgsConstructor
 `@Table`(uniqueConstraints = {
         `@UniqueConstraint`(
                 name = "uk_term_name",
                 columnNames = {"name"}
         )
 })
 public class Term {

src/main/java/com/example/solidconnection/term/domain/Term.java (1)

18-18: ⚠️ Potential issue | 🟠 Major

@AllArgsConstructor가 isCurrent 변환 로직을 우회할 수 있습니다.

1. @AllArgsConstructor는 Term(Long id, String name, Boolean isCurrent) 생성자를 자동 생성합니다.
2. 이 생성자는 Line 39의 isCurrent ? true : null 변환 로직을 거치지 않습니다.
3. 따라서 new Term(1L, "2025-1", false)를 호출하면 false가 그대로 DB에 저장되어, null 기반 UNIQUE 제약 설계 의도가 무너질 수 있습니다.

@AllArgsConstructor를 제거하거나, 테스트 전용이라면 접근 수준을 제한하는 것을 권장합니다.

♻️ 수정 제안

-@AllArgsConstructor

만약 테스트 등에서 all-args 생성자가 필요하다면, AccessLevel.PRIVATE으로 제한하세요:

-@AllArgsConstructor
+@AllArgsConstructor(access = AccessLevel.PRIVATE)